…ntext

- Ensures RangeVar receives correct objtype context to prevent 'ONLY' keyword
- Fixes ALTER TYPE RENAME ATTRIBUTE statements
- All 279 test suites now passing

Co-Authored-By: Dan Lynch <[email protected]>

…better readability

- Group (!context.parentNodeTypes.includes('AlterTypeStmt') && context.objtype !== 'OBJECT_TYPE') conditions
- Improves code semantics by making it clear both conditions relate to preventing ONLY keyword for ALTER TYPE operations
- Remove ternary operator from RenameStmt to maintain universal spawn() flow
- All 279 test suites passing

Co-Authored-By: Dan Lynch <[email protected]>

…xt parameters

- Update formatWindowFrame method signature to accept DeparserContext parameter
- Update deparseOperatorName method signature to accept DeparserContext parameter
- Replace all empty context creation with passed context in both methods
- Update all call sites to pass properly spawned context instead of creating empty contexts
- Ensures consistent context propagation in helper methods that are not actual node types

Co-Authored-By: Dan Lynch <[email protected]>

- Change visit() to pass original context instead of spawning new context
- Prevents premature context spawning and gives node methods control
- All tests continue to pass (279/279 test suites)
- Helper methods already properly use passed context parameters

Co-Authored-By: Dan Lynch <[email protected]>

…perty

- Add SqlFormatter as a property of DeparserContext with default instantiation
- Replace all this.formatter references with context.formatter throughout deparser
- Remove direct SqlFormatter import and private formatter property from Deparser class
- Update deparse() and visit() methods to create context with formatter when not provided
- Maintain modular design allowing external formatter instantiation and context passing
- Preserve existing formatting logic while improving context-aware formatting
- All 279 test suites continue to pass

Co-Authored-By: Dan Lynch <[email protected]>

- Make SqlFormatter private property on DeparserContext
- Add public methods: indent(), newline(), parens(), format(), isPretty()
- Replace all context.formatter.* references with direct context methods
- Fix CREATE TABLE constraint indentation using context.indentLevel
- UNIQUE and FOREIGN KEY constraints now properly indented relative to table structure
- Context spawning with increased indent levels for table elements
- All 279 test suites continue to pass

Co-Authored-By: Dan Lynch <[email protected]>

… indentLevel

- Remove explicit indentLevel increment when spawning table element contexts
- Let context.indent() method handle indentation automatically using indentLevel + 1
- UNIQUE and FOREIGN KEY constraints now properly indented relative to columns
- Constraints are visually distinct with proper logical grouping indentation
- All tests passing including CREATE TABLE pretty printing snapshots

Co-Authored-By: Dan Lynch <[email protected]>

- Replace hardcoded '  AND ' and '  OR ' spacing with context.indent() in BoolExpr
- Add proper indentLevel context spawning for GROUP BY and ORDER BY clauses
- Boolean expressions now use context-aware indentation for better logical grouping
- SELECT clause elements properly track indent levels through context spawning
- Consistent indentation management across nested statements and expressions

Co-Authored-By: Dan Lynch <[email protected]>

- Spawn context with increased indentLevel for CHECK constraint expressions
- Spawn context with increased indentLevel for nested SELECT statements
- Update snapshots to reflect proper indentation of logical scopes
- Ensure tax_rate expressions in CHECK constraints are properly indented
- Ensure nested SELECT FROM clauses are properly indented within subqueries

Co-Authored-By: Dan Lynch <[email protected]>

…, and nested SELECT statements

- Add indentToCurrentLevel() method to DeparserContext for precise current-level indentation
- Fix over-indentation in CHECK constraint expressions by using proper context spawning
- Improve CASE statement WHEN/ELSE clause alignment using indentToCurrentLevel()
- Resolve nested SELECT indentation issues in SubLink by managing indentLevel properly
- Fix BoolExpr AND/OR indentation to align with logical grouping
- All pretty printing tests now pass including constraints-9, selects-13, misc-8, and misc-9

Co-Authored-By: Dan Lynch <[email protected]>

…rrentLevel for proper systematic indentation

Co-Authored-By: Dan Lynch <[email protected]>

…and nested expressions

- Update WHERE clause to use context.indent() for proper first-line indentation
- Remove explicit indentLevel increment from WHERE context spawning
- All target test cases now pass: constraints-9, selects-13, misc-8, misc-9
- Indentation now follows systematic rules with indentLevel as contextual depth

Co-Authored-By: Dan Lynch <[email protected]>

- Change context.indentToCurrentLevel(tuple) to context.indent(tuple) for proper VALUES indentation
- Fixes INSERT statement indentation issues in casing-1, casing-10, casing-11, casing-12 tests
- Maintains successful fixes for constraints-9, selects-13, misc-8, misc-9 test cases
- Completes systematic indentation approach using indentLevel as contextual depth

Co-Authored-By: Dan Lynch <[email protected]>